Skip to content

Update Laravel API to use controller#1750

Merged
Pek5892 merged 7 commits into
devcode-it:masterfrom
Dasc3er:laravel-routing
Jul 20, 2026
Merged

Update Laravel API to use controller#1750
Pek5892 merged 7 commits into
devcode-it:masterfrom
Dasc3er:laravel-routing

Conversation

@Dasc3er

@Dasc3er Dasc3er commented Jan 24, 2026

Copy link
Copy Markdown
Contributor

Aggiornamento di API Platform per permettere l'utilizzo di Controller invece di Provider/Processor.
Usare i Controller permette maggiore controllo sul formato iniziale e finale dell'API, a scapito delle funzionalità avanzate di API Platform (Hydra IRI e simile).

Aggiunto supporto alla home di ApiPlatform con fix per gli assets al percorso <root OSM>/public/api-for-ui/api (serve login già effettuato).

Vantaggi:

  • Flessibilità su contenuto e metodo di mappatura (utilizzando Valinor come implementato, è più facile utilizzare DTO e gestire anche mappe particolari come DataTables che invece prima era incompleta)
  • Errori di validazione sono espliciti e possono essere personalizzati per chiarezza
  • Non è necessario registrare ogni provider/processor, quindi è più facile aggiungere contenuti dinamici e il bootstrap è più veloce

Svantaggi:

  • No funzionalità API Platform avanzate

  • Bisogna esplicare un casting per ogni controller adeguato per la DTO di input come per

      $data = $this->_cast($request, DataTablesLoadRequest::class);
    

@Dasc3er
Dasc3er force-pushed the laravel-routing branch 3 times, most recently from 588ec69 to e04a9d0 Compare February 7, 2026 08:50
@Dasc3er Dasc3er changed the title Update DataTables helper to use controller Update Laravel API to use controller Feb 7, 2026
@loviuz
loviuz requested a review from Pek5892 February 20, 2026 15:22
@devcode-it devcode-it deleted a comment from kilo-code-bot Bot Apr 20, 2026
@Dasc3er
Dasc3er force-pushed the laravel-routing branch from 0434c85 to 877b012 Compare July 7, 2026 18:25
@Dasc3er
Dasc3er force-pushed the laravel-routing branch 4 times, most recently from b3bb8ce to 4660f4a Compare July 11, 2026 12:04
Pek5892 added a commit that referenced this pull request Jul 20, 2026
- Added API integration tests for controller architecture
- Added DataTables deserialization tests
- Added Module permission tests
- Added comprehensive integration report
- Verified Valinor compatibility with PHP 8.3
- Confirmed DTO structure compatibility with DataTables JS
- Identified and documented fix requirements

Test Results:
✅ Dependencies: api-platform/laravel ^4.2.14, cuyz/valinor ^2.3
✅ Controller architecture: BaseController, DataTablesController
✅ Permissions: Module pivot handling, admin check
⚠️ DTO deserialization: Missing fromArray() methods (requires fix)
⚠️ DataTables JS: Type normalization needed (low priority)

Status: READY FOR MERGE with minimal fixes required
@Pek5892
Pek5892 merged commit a9d59cf into devcode-it:master Jul 20, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants